Ajout du unique_id au sensor d'énergie #296
Conversation
Permettrait possiblement d'arranger #281 Comme ils n'avaient pas de uniqueid il n'arrivaient pas à se populer. Les autres sensors (température, power, etc.) avait leur unique id et étaient fonctionnels.
|
oh wow facile de même! Good job @ic-dev21 |
There was a problem hiding this comment.
Est-ce qu'on serait pas mieux de mettre self._attr_name = f"Hilo Energy {device.name}" pour le nom de l'entité ?
J'aurais mis self._attr_unique_id = f"{slugify(device.name)}-energy" pour être comme les autres sensors mais ça va mettre en double tout les sensor d'energy de ceux qui ont deja fait la mise a jour donc je le changrais pas juste pour ça.
Penses-tu que c'est faisable de le lier avec le device? Dans la classe d'IntegrationSensor il y a la donnée device_info qui pourrait être utilisé. (Elle n'est présentement pas utilisé dans l'intégration)
There was a problem hiding this comment.
Ça me parait être une bonne suggestion par contre comme tu dis ça causerait du dédoublage.
As-tu la doc pas loin pour l’integration sensor? C’est pas fou comme idée.
There was a problem hiding this comment.
Le changement de nom ne fera pas de dédoublage mais je crois que le unique_id oui puisque les entité n'auront plus le même id.
Par contre pour je crois qu'il n'aura pas de dédoublage si on lie l'entité avec le device tout en gardant le même unique_id qui vient d'être ajouté.
J'ai pas trouver grand chose pour l’integration sensor à part sa définition dans le github de home assistant.
https://github.com/home-assistant/core/blob/dev/homeassistant/components/integration/sensor.py
There was a problem hiding this comment.
J’apprécie tes commentaires constructifs. Je suis pas un programmeur pour 2 cennes alors ça m’aide pas mal à m’améliorer!
Permettrait possiblement d'arranger #281
Comme ils n'avaient pas de uniqueid il n'arrivaient pas à se populer. Les autres sensors (température, power, etc.) avait leur unique id et étaient fonctionnels.
Merci de faire un release incluant une nouvelle version python-hilo pour inclure mon autre PR.